Skip to content

docs(github-mcp-server): sync template standards updates#346

Open
mberlofa wants to merge 2 commits into
mainfrom
docs/github-mcp-server-template-standards-sync
Open

docs(github-mcp-server): sync template standards updates#346
mberlofa wants to merge 2 commits into
mainfrom
docs/github-mcp-server-template-standards-sync

Conversation

@mberlofa

@mberlofa mberlofa commented Jul 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

  • document optional ingressClassName rendering and NetworkPolicy extra egress for GitHub MCP Server
  • enable the existing GitHub MCP Server playground config in the playground registry
  • expose an optional NetworkPolicy extra egress playground field

Validation

  • make site-sync-check CHART=github-mcp-server
  • npm run lint
  • npm run format:check
  • npm run build
  • make release-check REPO=site
  • make attribution-check REPO=site

Charts PR: helmforgedev/charts#668
Issue: helmforgedev/charts#633

@mberlofa mberlofa mentioned this pull request Jul 3, 2026
Open
@coderabbitai

coderabbitai Bot commented Jul 3, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

This PR updates olivetin playground configuration with an empty Ingress Class option and a new NetworkPolicy section, documents github-mcp-server ingress and network policy behavior, and adds github-mcp-server to the playground sync allowlist.

Changes

Playground config and documentation updates

Layer / File(s) Summary
Olivetin playground config NetworkPolicy and Ingress Class options
src/data/playground-configs.ts		 Adds an empty option to the Ingress Class select and inserts a new NetworkPolicy section with a DNS egress port field defaulting to 53.
github-mcp-server docs and playground sync entry
src/pages/docs/charts/github-mcp-server.mdx, src/pages/playground.astro		 Expands persistence and exposure documentation for github-mcp-server, and adds the chart to siteSyncPlaygroundConfigs.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly matches the PR’s main theme: syncing GitHub MCP Server docs and playground config with template standards updates.
Description check ✅ Passed The description includes a summary, validation steps, and the related issue; it only omits the template's exact Quality Gates checklist format.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/github-mcp-server-template-standards-sync

Comment @coderabbitai help to get the list of available commands.

@mberlofa mberlofa mentioned this pull request Jul 3, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mberlofa